Skip to content

Conversation

mateuszjasiuk
Copy link
Collaborator

@mateuszjasiuk mateuszjasiuk commented Aug 20, 2025

  • replaces imports
  • fixes tests
  • removes all the wasm build code
  • removes shared, crypto and sdk packages

@mateuszjasiuk mateuszjasiuk force-pushed the feat/use-namada-sdkjs-in-extension branch 2 times, most recently from c8c4baf to 4a4ed2b Compare August 20, 2025 13:36
@mateuszjasiuk mateuszjasiuk marked this pull request as ready for review August 21, 2025 09:26

// SDK wasm init must be called
await initWasm();
// We only do this to initialize wasm, props don't matter
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a note, though I think this is fine: There is an init that is directly exported (same with */inline) that doesn't initialize Sdk, but since we don't use the inline imports in extension, you'd have to pass the wasm as an arg :D Maybe that can be improved in the future, but let's keep it like this for now!

@mateuszjasiuk mateuszjasiuk force-pushed the feat/use-namada-sdkjs-in-extension branch from 5261d8d to 1be9ce3 Compare August 21, 2025 10:13
Copy link
Collaborator

@jurevans jurevans left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, LGTM!

@mateuszjasiuk mateuszjasiuk merged commit 9391b05 into main Aug 21, 2025
6 checks passed
@mateuszjasiuk mateuszjasiuk deleted the feat/use-namada-sdkjs-in-extension branch August 21, 2025 10:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants